Repository navigation
chore(agents): trim root CLAUDE.md by ablation; AGENTS.md links to it - #9745
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
[Medium impact] This PR appears safe to merge. SummaryTrims root agent guidance and makes
Reviews (3) · Last reviewed commit: "chore(agents): keep auth-ui off the emcn..." · Reviewed by Greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 9 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
CLAUDE.mdfrom 22.6KB to 11.5KB (-49%) by treating it as an ablation: traced every section to the commit that added it, bucketed each rule, then replayed tasks against old vs new guidancetools.config.paramscoercion, testing culture, the local gate); rules already enforced by acheck:*audit or biome become one-line pointers (the audit message names the fix); narrow area rules move to their path-scoped rule file; the repo tree and generic coaching are goneCommon utilitieskeeps the ID rule (namedrandomUUIDimports slip pastcheck:utils) and the helperscheck:utilscan't catch (toArray, scalar coercions,compareStrings,backoffWithJitter/parseRetryAfter), with their typed-source and stricter-local-helper exceptionsAGENTS.mdis now a symlink toCLAUDE.md(same asapps/sim/CLAUDE.md->AGENTS.md). Drops the Turborepo agent block: the installed turbo 2.9.14 ships nodocs/it points at and never writes the blocksim-styling.mdnow also loads forpackages/workflow-renderertsx, andsim-testing.mdforapps/sim/scripts/test-*-e2e.ts, so the rules that left the root still reach those files.claude/rules/sim-styling.md(loads for everyapps/sim/**/*.tsx); two skills that citedCLAUDE.md "Styling and EMCN"now point thereType of Change
Testing
check:utils,check:source-conventions,check:client-boundary,check:file-names,check:explicit-any,check:comment-hygiene; neither variant wrote after-the-fact unit testspaths:cover the files it matters for, a skill the root points to, or an audit whose failure message states the fix). A separate blind review against the old file found 12 gaps (dropped exceptions, softened wording, rule files scoped narrower than the root rule); all are restored in the second commitcheck:guidance-refs(78 docs, everyCLAUDE.md "<Section>"citation still resolves),check:skills,bun run lintChecklist
test-auditauthoring gate)🤖 Generated with Claude Code